-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update cr.yml #36
Update cr.yml #36
Conversation
GPT 리뷰 라벨 조건을 삭제합니다. 앞으로 리뷰를 받아보고 많다고 생각되면 라벨 조건을 추가하도록 해요.
Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information |
runs-on: ubuntu-latest | ||
steps: | ||
- uses: anc95/ChatGPT-CodeReview@main | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }} | ||
# optional | ||
LANGUAGE: Korean | ||
PROMPT: 당신은 10년차 시니어 개발자입니다. 우리가 작성한 코드에 문제가 없는지 리뷰해주세요. 대답은 한국어로 작성해주시고 단순한 부분이나 큰 이슈가 없는 부분은 리뷰해주지 않아도 되요. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코드 리뷰를 보면, 주석을 통해 사용 가능한 옵션 중 하나인 "if" 구문이 주석 처리되어 있습니다. "if" 구문은 제거하시는 것이 좋겠습니다. 이 부분만 제외하면 큰 문제는 없는 것으로 보입니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
와우 리뷰 달아주네요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reopen 의 경우에도 리뷰하도록 추가한 점 확인하였습니다.
LGTM
확인했습니다! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저도 확인했습니다~!
GPT 리뷰 라벨 조건을 삭제합니다. 앞으로 리뷰를 받아보고 많다고 생각되면 라벨 조건을 추가하도록 해요.
💡 Motivation and Context
여기에 왜 이 PR이 필요했는지, PR을 통해 무엇이 바뀌는지에 대해서 설명해 주세요
🔨 Modified
🌟 More
📋 커밋 전 체크리스트
🤟🏻 PR로 완료된 이슈
closes #